Skip to content

Add Ahoy tracking to FAQ searches and views#1315

Merged
maebeale merged 1 commit intomainfrom
maebeale/faq-search-tracking
Mar 8, 2026
Merged

Add Ahoy tracking to FAQ searches and views#1315
maebeale merged 1 commit intomainfrom
maebeale/faq-search-tracking

Conversation

@maebeale
Copy link
Collaborator

@maebeale maebeale commented Mar 3, 2026

What is the goal of this PR and why is this important?

  • FAQ searches and page views were not being captured as Ahoy events
  • Every other public-facing controller (Resources, Workshops, Tutorials, Events, etc.) already tracks these, so FAQs were a gap in analytics

How did you approach the change?

  • Added include AhoyTracking to FaqsController
  • Added track_index_intent(Faq, @faqs, params) in the index action to capture search queries and result counts
  • Added track_view(@faq) in the show action to capture individual FAQ views
  • Added request specs verifying the tracking calls

UI Testing Checklist

  • Visit /faqs and verify page still loads
  • Search for a FAQ and verify results still appear
  • Click into a FAQ and verify the show page loads

Anything else to add?

  • Follows the exact pattern used in ResourcesController, WorkshopsController, TutorialsController, etc.
  • No UI changes

🤖 Generated with Claude Code

FaqsController was missing the AhoyTracking concern, so FAQ searches
and page views were not being captured as Ahoy events. This adds
track_index_intent on index and track_view on show, matching the
pattern used in other controllers.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@maebeale maebeale changed the title Add Ahoy tracking to FAQ searches and views HOLD: Add Ahoy tracking to FAQ searches and views Mar 4, 2026
@maebeale maebeale changed the title HOLD: Add Ahoy tracking to FAQ searches and views Add Ahoy tracking to FAQ searches and views Mar 8, 2026
@maebeale maebeale merged commit 31927b5 into main Mar 8, 2026
3 checks passed
@maebeale maebeale deleted the maebeale/faq-search-tracking branch March 8, 2026 12:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant